Skip to content

Conversation

@daniel-lxs
Copy link
Member

@daniel-lxs daniel-lxs commented Nov 4, 2025

Fixes dimension mismatch causing 'Vector dimension error: expected dim: 3072, got 1536' when using OpenRouter provider with mistralai/codestral-embed-2505 model.

  • Updated model profile dimension to match actual API response (1536)
  • Fixed related test cases
  • Enables automatic Qdrant collection recreation with correct dimensions

Important

Fixes dimension mismatch for OpenRouter Mistral model, ensuring correct dimension handling and updating related tests.

  • Behavior:
    • Fixes dimension mismatch for mistralai/codestral-embed-2505 model in embeddingModels.ts from 3072 to 1536.
    • Ensures correct dimension handling for OpenRouter provider, preventing 'Vector dimension error'.
    • Enables automatic Qdrant collection recreation with correct dimensions.
  • Tests:
    • Updates test cases in config-manager.spec.ts to verify correct dimension handling across restarts.
    • Adds tests to ensure no restart is required when model dimensions remain the same.
    • Adds tests to ensure restart is required when model dimensions change.

This description was created by Ellipsis for 5537dd0. You can customize this summary. It will automatically update as commits are pushed.

Fixes dimension mismatch causing 'Vector dimension error: expected dim: 3072, got 1536'
when using OpenRouter provider with mistralai/codestral-embed-2505 model.

- Updated model profile dimension to match actual API response
- Fixed related test cases to use correct dimension
@daniel-lxs daniel-lxs requested review from cte, jr and mrubens as code owners November 4, 2025 17:21
@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Nov 4, 2025
@roomote
Copy link

roomote bot commented Nov 4, 2025

See this task on Roo Code Cloud

Review complete. No issues found.

The dimension correction from 3072 to 1536 is correct and matches the actual API response. Test coverage is comprehensive.

Mention @roomote in a comment to trigger your PR Fixer agent and make changes to this pull request.

@dosubot dosubot bot added the bug Something isn't working label Nov 4, 2025
@daniel-lxs daniel-lxs moved this from Triage to PR [Needs Review] in Roo Code Roadmap Nov 4, 2025
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Nov 4, 2025
@mrubens mrubens merged commit f717863 into main Nov 4, 2025
20 checks passed
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Nov 4, 2025
@github-project-automation github-project-automation bot moved this from PR [Needs Review] to Done in Roo Code Roadmap Nov 4, 2025
@mrubens mrubens deleted the fix/openrouter-mistral-dimension branch November 4, 2025 17:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working lgtm This PR has been approved by a maintainer size:L This PR changes 100-499 lines, ignoring generated files.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants